-
Notifications
You must be signed in to change notification settings - Fork 43
Add support for discovery extensions #760
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
83825d3
to
a9277d8
Compare
87a764c
to
27e0ae6
Compare
Currently, the implementation requires returning the absolute path to a resource manifest - but I Right now, it expects JSON like this: {"resourceManifestPath":"/foo/bar/baz.dsc.resource.json"} But we could have it return this, and parse the last segment of the path to determine kind: {"absoluteManifestPath":"/foo/bar/baz.dsc.extension.json"} Or require it return either If we support extensions discovering extensions, we would need to recursively handle extension |
I think we can defer extension discovery til when we need it |
e1cdb55
to
233c3f5
Compare
Renamed from |
233c3f5
to
014a16f
Compare
PR Summary
dsc
to list extensions:dsc extension list
discover
capability currently supported.The output from a discovery extension must match:
PR Context
Fix #681